-
Notifications
You must be signed in to change notification settings - Fork 619
[Docs] Remove Nebula from portal navigation #7576
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Docs] Remove Nebula from portal navigation #7576
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
WalkthroughReferences to the "Nebula" feature have been removed from the application's header navigation and from two sections on the main page. This includes deleting "Nebula" links, icons, and cards, as well as updating an icon for the "Vault" article card. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Header
participant MainPage
User->>Header: Loads navigation
Header-->>User: Displays links (without "Nebula")
User->>MainPage: Loads API Reference & Learning Resources sections
MainPage-->>User: Displays sections (without "Nebula" cards)
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
💤 Files with no reviewable changes (1)
🚧 Files skipped from review as they are similar to previous changes (1)
✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7576 +/- ##
=======================================
Coverage 56.51% 56.51%
=======================================
Files 906 906
Lines 57992 57992
Branches 4223 4223
=======================================
Hits 32777 32777
Misses 25107 25107
Partials 108 108
🚀 New features to boost your workflow:
|
size-limit report 📦
|
Merge activity
|
``` <!-- ## title your PR with this format: "[Portal] Feature: Remove Nebula from portal navigation" ## Notes for the reviewer This PR removes all user-facing references to Nebula from the portal navigation and homepage, as requested. - **Key point:** All Nebula pages and underlying content remain untouched; only navigation links and homepage sections were removed. - Removed Nebula from the main header navigation and the APIs dropdown. - Removed Nebula sections from the homepage (API Reference and Learning Resources). - Updated the Vault section icon on the homepage from `NebulaIcon` (which was removed) to `ConnectIcon`. ## How to test 1. Run the portal locally. 2. Verify "Nebula" is no longer present in the main navigation bar or the "APIs" dropdown. 3. On the homepage (`/`), confirm that the "Nebula" cards are removed from the "API Reference" and "Learning Resources" sections. 4. Confirm that the "Vault" card in the "Learning Resources" section now uses the `ConnectIcon`. 5. Run `npm run portal:build` to ensure the project builds successfully. --> ``` --- [Slack Thread](https://thirdwebdev.slack.com/archives/C04DYC3G5NZ/p1752094214240499?thread_ts=1752094214.240499&cid=C04DYC3G5NZ) <!-- start pr-codex --> --- ## PR-Codex overview This PR primarily focuses on removing references to `Nebula` from the application, including links and UI components, while retaining and emphasizing `Insight` and `Vault`. ### Detailed summary - Removed `Nebula` link from `connectLinks` in `Header.tsx`. - Deleted `Nebula` API reference card from `ReferenceSection` in `page.tsx`. - Eliminated `Nebula` card from `LearningResourcesSection` in `page.tsx`. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * None. * **Bug Fixes** * None. * **Refactor** * Removed all references to the "Nebula" feature from navigation menus and resource sections. * Updated the icon for the "Vault" article card. * **Chores** * Cleaned up unused imports related to "Nebula". <!-- end of auto-generated comment: release notes by coderabbit.ai -->
4c46050 to
381a282
Compare
Slack Thread
PR-Codex overview
This PR focuses on removing references to
Nebulafrom theHeader.tsxandpage.tsxfiles, while keepingInsightandVault. It updates the navigation links and card components accordingly.Detailed summary
Nebulalink fromconnectLinksinHeader.tsx.NebulaAPI reference card inReferenceSectionofpage.tsx.Nebulafrom theLearningResourcesSectionand replaced its icon withConnectIconforVault.Summary by CodeRabbit
New Features
Bug Fixes
Refactor
Chores